Skip to content

Fix build-time GraalVmProcessor warning#1801

Merged
WouterGritter merged 1 commit into
PaperMC:dev/4.0.0from
WouterGritter:fix-proxy-build-warning
Jul 11, 2026
Merged

Fix build-time GraalVmProcessor warning#1801
WouterGritter merged 1 commit into
PaperMC:dev/4.0.0from
WouterGritter:fix-proxy-build-warning

Conversation

@WouterGritter

@WouterGritter WouterGritter commented May 21, 2026

Copy link
Copy Markdown
Member

Building the project shows the following warning:

> Task :velocity-proxy:compileJava
warning: The `GraalVmProcessor` annotation processor is missing the recommended `log4j.graalvm.groupId` and `log4j.graalvm.artifactId` options.
  To follow the GraalVM recommendations, please add the following options to your build tool:
    -Alog4j.graalvm.groupId=<groupId>
    -Alog4j.graalvm.artifactId=<artifactId>

This PR fixes this by adding the required options.

WouterGritter
WouterGritter commented May 21, 2026
View reviewed changes
Comment thread proxy/build.gradle.kts
@@ -112,6 +112,15 @@ tasks {
workingDir = file("run").also(File::mkdirs)
standardInput = System.`in` // Doesn't work?

@WouterGritter WouterGritter May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

offtop: // Doesn't work? lol

@WouterGritter WouterGritter changed the base branch from dev/3.0.0 to dev/4.0.0 July 11, 2026 19:55
@WouterGritter WouterGritter merged commit 729a050 into PaperMC:dev/4.0.0 Jul 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@electronicboy electronicboy electronicboy approved these changes
+1 more reviewer
@R00tB33rMan R00tB33rMan R00tB33rMan approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@WouterGritter @electronicboy @R00tB33rMan